Skip to content

Test: Count a TSan report only when the racing access is in the extension - #230

Merged
b-re-w merged 1 commit into
developfrom
feat/tp-tsan-access-frame
Oct 17, 2026
Merged

b-re-w merged 1 commit into
developfrom
feat/tp-tsan-access-frame

Conversation

@b-re-w

@b-re-w b-re-w commented Oct 17, 2026

Copy link
Copy Markdown
Member

The first-report output added in #221 showed what failed on Linux CI: two PyUnicode_FromString memcpys in libpython, writing into memory that the uninstrumented allocator had handed from one finished thread to another. TSan cannot see that synchronisation. Our tp_wrap_usek only appeared further up as the caller. The old filter counted any report with an extension frame anywhere in it.

Now a report counts only when the access itself was made by the extension: the first frame of either racing access outside the sanitizer runtime must be in ftmod.

Verification:

@b-re-w
b-re-w merged commit 71a7a03 into develop Oct 17, 2026
@b-re-w
b-re-w deleted the feat/tp-tsan-access-frame branch October 17, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant